feat: 리액션 API 추가 및 추천리스트 API, 리스트 상세조회 API 재구현 #304

pparkjs · 2024-09-29T08:28:02Z

Description

🛠️변경사항

멋져요, 공감해요, 감사해요라는 리액션을 추가함
-> 언제든 리액션에 추가가 필요할 경우 쉽게 추가할 수 있도록 설계함
-> 추후 리액션 카운트 업데이트에 대한 동시성 처리를 할 예정임
리액션 추가에 의한 리스트 상세정보 API 수정함
-> 리스트 1번에 대한 어떠한 리액션을 누군가 최초로 했을 경우 그 순간 reaction_stats 테이블에 데이터가 insert 되도록 함
이를 통해 데이터 메모리 낭비를 방지하였음
기존 트랜딩리스트 -> 추천리스트로 명칭이 바뀌었고 이에따라 추천리스트의 정책이 변경됨에 따라 API를 수정함
-> 가장 최근에 30일간 리액션이 달렸으며 리액션 횟수의 합이 가장 높은 리스트 TOP10
만약 리액션 횟수가 같은 경우 리스트 수정일이 빠른 리스트 가져오도록 변경됨

🚨 Merge하기 전 주의!

dev와 prod에 merge 하기 전 꼭 필수로 쿼리를 실행해 줘야함

create table reaction_stats
(
    count        int                              not null,
    created_date datetime(6)                      null,
    id           bigint auto_increment
        primary key,
    list_id      bigint                           not null,
    updated_date datetime(6)                      null,
    reaction     enum ('COOL', 'AGREE', 'THANKS') not null,
    constraint reaction_stats_list_fk
        foreign key (list_id) references list (id)
);

create table user_reaction
(
    created_date datetime(6)                      null,
    id           bigint auto_increment
        primary key,
    list_id      bigint                           not null,
    user_id      bigint                           not null,
    reaction     enum ('COOL', 'AGREE', 'THANKS') not null,
    constraint UniqueIdAndUserIdAndReaction
        unique (id, user_id, reaction),
    constraint user_reaction_list_fk
        foreign key (list_id) references list (id)
);

Relation Issues

close 리액션 API 추가 및 이에 따른 리스트 상세 조회, 트랜딩 리스트 조회 API 수정 #298

kdkdhoho

복잡했을텐데 구현 깔끔하네요!
일단 몇 가지 제안 드려봤습니다.

그런데 리스트 상세 조회 시 최신 댓글 정보 가져오는 변경 사항을 먼저 머지하면, 일부 수정하셔야 해서 그때 다시 보겠습니다.

고생하셨어요!

src/main/java/com/listywave/list/application/domain/reaction/Reaction.java

src/main/java/com/listywave/list/application/domain/reaction/ReactionStats.java

src/main/java/com/listywave/list/application/domain/reaction/UserReaction.java

src/main/java/com/listywave/list/application/service/ListService.java

src/main/java/com/listywave/list/presentation/controller/ListController.java

kdkdhoho · 2024-10-11T06:40:20Z

src/test/java/com/listywave/acceptance/list/ListAcceptanceTest.java

+    @Nested
+    class 리액션_테스트 {


리액션은 리스트에서 분리하는 건 어떠신가요??

좋습니다!

kdkdhoho · 2024-10-11T06:43:44Z

src/main/java/com/listywave/list/presentation/controller/ListController.java

+    @PostMapping("/lists/{listId}/reaction")
+    public ResponseEntity<String> handleReaction(
+            @Auth Long loginUserId,
+            @PathVariable("listId") Long listId,
+            @RequestBody ReactionRequest request
+    ) {
+        listService.handleReaction(loginUserId, listId, request.reaction());
+        return ResponseEntity.noContent().build();
+    }


아 그리고 리액션은 List랑 패키지 분리하는 건 어떠신가요??

list에서만 사용해서 포함시키긴 했는데 아무래도 따로 분리하는게 나을 거 같죠?

src/main/java/com/listywave/list/application/domain/reaction/ReactionStats.java

pparkjs

@kdkdhoho 답변 부탁드립니다 ㅎㅎ

src/main/java/com/listywave/list/application/domain/reaction/Reaction.java

src/main/java/com/listywave/list/application/domain/reaction/ReactionStats.java

src/main/java/com/listywave/list/application/domain/reaction/UserReaction.java

src/main/java/com/listywave/list/presentation/controller/ListController.java

pparkjs · 2024-10-17T23:06:49Z

src/test/java/com/listywave/acceptance/list/ListAcceptanceTest.java

+    @Nested
+    class 리액션_테스트 {


좋습니다!

pparkjs · 2024-10-17T23:07:58Z

src/main/java/com/listywave/list/presentation/controller/ListController.java

+    @PostMapping("/lists/{listId}/reaction")
+    public ResponseEntity<String> handleReaction(
+            @Auth Long loginUserId,
+            @PathVariable("listId") Long listId,
+            @RequestBody ReactionRequest request
+    ) {
+        listService.handleReaction(loginUserId, listId, request.reaction());
+        return ResponseEntity.noContent().build();
+    }


list에서만 사용해서 포함시키긴 했는데 아무래도 따로 분리하는게 나을 거 같죠?

pparkjs · 2024-10-27T04:20:15Z

복잡했을텐데 구현 깔끔하네요! 일단 몇 가지 제안 드려봤습니다.

그런데 리스트 상세 조회 시 최신 댓글 정보 가져오는 변경 사항을 먼저 머지하면, 일부 수정하셔야 해서 그때 다시 보겠습니다.

고생하셨어요!

반영 했습니다! 그 Unique key에서 id는 기존 unique 고유 특성 있어서 빼도 된다고 하셨었는데
뺴지 않은 이유는 저렇게 multi로 Unique Key로 한 경우 3개의 unique key가 같은 경우만 위배되도록 하기 위함인데 만약 id를 빼게되면
리스트ID가 1, 2 인 것에 박정수 사용자가 고마워요를 각각 달면 위배되게 됩니다! 실제로 테스트에서 그렇게 결과가 나옴
그래서 id까지 포함시켰습니다!

kdkdhoho

풍부한 테스트코드와 유니크 제약 조건 테스트까지.. 넘 수고하셨습니다 !
변수명 하나만 제안드립니다 ㅎ
Approve 할게요 !!

kdkdhoho · 2024-10-27T05:47:55Z

src/main/java/com/listywave/reaction/application/domain/Reaction.java


-    private final String name;
+    private final String displayName;


혹시 viewName은 어떠신가요..? ㅎ
CategoryType에서 viewName을 사용하고 있는데 통일시키는게 좋을 것 같아서요 !!

pparkjs added 기능 테스트 labels Sep 29, 2024

pparkjs self-assigned this Sep 29, 2024

pparkjs requested a review from kdkdhoho as a code owner September 29, 2024 08:28

pparkjs linked an issue Sep 29, 2024 that may be closed by this pull request

리액션 API 추가 및 이에 따른 리스트 상세 조회, 트랜딩 리스트 조회 API 수정 #298

Closed

3 tasks

kdkdhoho reviewed Oct 11, 2024

View reviewed changes

pparkjs commented Oct 19, 2024

View reviewed changes

pparkjs added 3 commits October 27, 2024 11:31

feat: 리액션 API 추가 및 추천리스트 API, 리스트 상세조회 API 재구현

537edc1

test: 리액션 관련 테스트 추가 및 이에 따른 변경된 API에 대한 테스트코드 수정

e1783ed

refactor: 코드리뷰에 의한 네이밍 변경 및 리액션 패키지구조 별도 추출

4b443a3

pparkjs force-pushed the feat/298 branch from c592a4b to 4b443a3 Compare October 27, 2024 04:13

kdkdhoho approved these changes Oct 27, 2024

View reviewed changes

refactor: 코드리뷰에 의한 네이밍 변경

46147bf

pparkjs merged commit ef84178 into dev Oct 27, 2024
1 check passed

pparkjs deleted the feat/298 branch October 27, 2024 11:02

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: 리액션 API 추가 및 추천리스트 API, 리스트 상세조회 API 재구현 #304

feat: 리액션 API 추가 및 추천리스트 API, 리스트 상세조회 API 재구현 #304

pparkjs commented Sep 29, 2024 •

edited

Loading

kdkdhoho left a comment

kdkdhoho Oct 11, 2024

pparkjs Oct 17, 2024

kdkdhoho Oct 11, 2024

pparkjs Oct 17, 2024

pparkjs left a comment

pparkjs Oct 17, 2024

pparkjs Oct 17, 2024

pparkjs commented Oct 27, 2024

kdkdhoho left a comment

kdkdhoho Oct 27, 2024

feat: 리액션 API 추가 및 추천리스트 API, 리스트 상세조회 API 재구현 #304

feat: 리액션 API 추가 및 추천리스트 API, 리스트 상세조회 API 재구현 #304

Conversation

pparkjs commented Sep 29, 2024 • edited Loading

Description

🛠️변경사항

🚨 Merge하기 전 주의!

Relation Issues

kdkdhoho left a comment

Choose a reason for hiding this comment

kdkdhoho Oct 11, 2024

Choose a reason for hiding this comment

pparkjs Oct 17, 2024

Choose a reason for hiding this comment

kdkdhoho Oct 11, 2024

Choose a reason for hiding this comment

pparkjs Oct 17, 2024

Choose a reason for hiding this comment

pparkjs left a comment

Choose a reason for hiding this comment

pparkjs Oct 17, 2024

Choose a reason for hiding this comment

pparkjs Oct 17, 2024

Choose a reason for hiding this comment

pparkjs commented Oct 27, 2024

kdkdhoho left a comment

Choose a reason for hiding this comment

kdkdhoho Oct 27, 2024

Choose a reason for hiding this comment

pparkjs commented Sep 29, 2024 •

edited

Loading